Skip to content

Rebase#11

Merged
Jeck0v merged 12 commits intofeat/enhanced-errorsfrom
main
Sep 22, 2025
Merged

Rebase#11
Jeck0v merged 12 commits intofeat/enhanced-errorsfrom
main

Conversation

@Jeck0v
Copy link
Copy Markdown
Owner

@Jeck0v Jeck0v commented Sep 22, 2025

No description provided.

Jeck0v and others added 12 commits September 15, 2025 14:49
the test on line 327 in `src/athena/generator/compose.rs` is checking
that the generated YAML does not contain a `version:` field:

```rust
assert!(!yaml.contains("version:"));
```
This test is actually validating that the version field has been
properly removed from the Docker Compose output.
 Enhanced test structure:
- 67 integration tests organized by functionality (CLI, Docker
generation, boilerplate, structural)
- Lightweight structural tests focusing on logic over format (no heavy
snapshot tests)
  - Modular boilerplate tests by framework (FastAPI, Go, Flask)
 

Addition of documentation and modification of the readme file
```
  docs/
  ├── INSTALLATION.md     # Complete installation guide (Makefile, Windows, manual)
  ├── DSL_REFERENCE.md    # Complete DSL syntax and directives reference
  ├── EXAMPLES.md         # Practical .ath file examples
  ├── BOILERPLATE.md      # Project generator documentation
  ├── ARCHITECTURE.md     # System design and components
  ├── DEVELOPMENT.md      # Contributing and development workflow
  └── TESTING.md          # Comprehensive test suite documentation
  ```
1. Port conflict detection `compose.rs`
    - Automatically detects duplicate host ports between services
    - Clear error messages with alternative port suggestions
2. Comprehensive integration tests
`tests/integration/error_handling_test.rs`
    - 4 new integration tests following the documented structure
    - Tests via CLI with `assert_cmd` and `predicates`
    - Fixture files created in `tests/fixtures/`
3. Maintained unit tests - compose.rs:tests
- Tests for the `extract_host_port()` and `generate_port_suggestions()`
utility functions
    - Basic generation test maintained
@Jeck0v Jeck0v merged commit 4969c02 into feat/enhanced-errors Sep 22, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant